Skip to content
This repository was archived by the owner on Sep 28, 2022. It is now read-only.

Doc update#129

Merged
yuce merged 5 commits intoFeatureBaseDB:masterfrom
yuce:doc-update
Oct 30, 2018
Merged

Doc update#129
yuce merged 5 commits intoFeatureBaseDB:masterfrom
yuce:doc-update

Conversation

@yuce
Copy link
Contributor

@yuce yuce commented Oct 25, 2018

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 4e4993d on yuce:doc-update into ccd6314 on pilosa:master.

@coveralls
Copy link

coveralls commented Oct 25, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 2a58e88 on yuce:doc-update into ccd6314 on pilosa:master.

@yuce yuce changed the title Doc update [WIP] Doc update Oct 25, 2018
@yuce yuce changed the title [WIP] Doc update Doc update Oct 29, 2018
@yuce yuce requested a review from alanbernstein October 29, 2018 12:46
pilosa/orm.py Outdated
"""Valid time quantum values.

* See: `Data Model <https://www.pilosa.com/docs/data-model/>`_
* See: `Data Model/Time Quantum <https://www.pilosa.com/docs/latest/data-model/#time-quantum/>`_
Copy link
Contributor

@alanbernstein alanbernstein Oct 29, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove trailing slash on these urls to get anchor links to work when copy+pasting

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated

Copy link
Contributor

@alanbernstein alanbernstein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@yuce yuce merged commit 04230ae into FeatureBaseDB:master Oct 30, 2018
@yuce yuce deleted the doc-update branch October 30, 2018 08:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants